button: Split out function

This is in preparation of the following cleanup commits.

button: Don't emit a signal on broken grabs

We don't emit one on grab notify, so we don't want one for broken grabs
either.

gdkwindow: Remove an unused assignment

The show variable is assigned to in both branches of an if block a
little further down.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gdkwindow: Remove an unused assignment

scale is only used if HAVE_CAIRO_SURFACE_SET_DEVICE_SCALE is defined.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintunixdialog: Remove a redundant (GFile != NULL) check

g_file_new_for_uri() is guaranteed to return a non-NULL value, so this
check was redundant, and was confusing the static analyser into
returning a false positive, where it thought the file could be NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Eliminate a dead assignment

This is technically a dead assignment, but is nice to retain for
clarity. Moving it to the variable definition shuts scan-build up.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkthemingengine: Eliminate a dead assignment

This is technically a dead assignment, but is nice to retain for
clarity. Moving it to the variable definition shuts scan-build up.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkthemingengine: Fix a definite use of an uninitialised variable

At this point, segments[1] is always uninitialised, and is used to
initialise itself. Looking at the code in the branch above, this appears
to have been a typo from segments[0], as segments[1] seems to typically
be 2 * segments[0].

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkpapersize: Reformat #if preprocessor commands

This eliminates some false positive warnings from scan-build, which was
not interpreting the #ifs and hence warning about unbalanced #endifs.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkmenu: Fix potential use of uninitialised variables

The child_height out variable is only valid if compute_child_offset()
returns TRUE.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeviewaccessible: Fix potential uses of uninitialised variables

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

GtkRevealer: Fix size allocation with disabled animation

When animation is disabled, we use 'none' as the effective
transition type. So far, this transition type failed to change
the size request, causing the revealer to always take up the
space of the child, even when the child is not shown.

https://bugzilla.gnome.org/show_bug.cgi?id=719568

Fix the AtkImage implementation for GtkIconView items

We were returning uninitialized values for image size and
position.

Found in a pile of patches in
https://bugzilla.gnome.org/review?bug=712760

Code cleanup

Don't compare booleans to FALSE.

annotations: Set GtkTreeModelFilterModifyFunc value argument to out

Use (out caller-allocates) annotation for the "value" argument to
GtkTreeModelFilterModifyFunc. This is needed because GI based language
bindings coerce GValue input args into native types and there is no
opportunity to set the value within the GValue itself.

https://bugzilla.gnome.org/show_bug.cgi?id=719460

Add class toolbar before the style context is created for the widget.

The call to gtk_button_set_relief() in gtk_toolbar_init() indirectly
used the style context of the half-created widget, before we had a
chance to add the "toolbar" style class to it.

Reorder gtk_toolbar_init() to ensure that the proper style class is
set first.

https://bugzilla.gnome.org/show_bug.cgi?id=719595

infobar: Fix visibility when enable-animations=false

When enable-animations is false, the revealer's child-revealed property is
notified immediately, so make sure to connect to it before toggling the
revealer.

https://bugzilla.gnome.org/show_bug.cgi?id=719510

Correct the deprecation annotation for gtk_widget_modify_bg

It was pointing to a nonexisting function.

https://bugzilla.gnome.org/show_bug.cgi?id=719469

Don't use depeprecated api

GtkPrintUnixDialog and testtitlebar were still using
gtk_widget_set_margin_left/right. Switch them over to
gtk_widget_set_margin_start/end.

message dialog: Make labels wrap at a reasonable length

Several labels in GtkMessageDialog were set to wrap, but
did not set max-width-chars. Fix that.

about dialog: Make labels wrap at reasonable length

Several labels in GtkAboutDialog are set to wrap, but
don't set a value for max-width-chars. Fix that.

places sidebar: Use a symbolic desktop icon

This was pointed out in

https://bugzilla.gnome.org/show_bug.cgi?id=719505

build/win32/vs*/README.txt: Re-word a bit

Use PATH rather than path to make things clearer

GtkAboutDialog: fixed calling gtk_about_dialog_set_logo_icon_name()
with a NULL icon_name. Now it will actually use the default
window icon set as mention in the docs

GtkIconCache: find_image_offset() return 0 if icon_name is NULL.
This avoid a crash calling gtk_about_dialog_set_logo_icon_name() with a NULL icon_name.

Fix CSD check for Win32

Win32 does not have alpha channel currently ; fix the check
for this, so trying to enable CSDs on this platform will
not "succeed" and crash the app anymore.

Partially fixes gtk3-widget-factory.

Visual C++ Projects: Add rules to generate GTK DBus sources

Since commit 7c2a5072 the gtkdbusgenerated.[c|h] are not included in the
dist tarball and thus have to be generated, which broke the Visual C++
builds.

This patch adds property sheets and custom build rules for the Visual C++
projects so that gtkdbusgenerated.[c|h] will be generated upon building the
GTK+ DLL sources.

This also tells people building GTK+ from the projects that they need to
have Python 2/3 installed and the Python interpretor needs to be in their
PATH before building GTK+ from the projects.

Update GDK-Broadway MSVC Projects

-Improve optimization a bit for broadwayd, by enabling link time code
 generation
-Add PlatformToolset tag for the Visual C++ 2010 projects, to ease
 transition to Visual C++ 2012/2013

Tamil Translations Updated

Don't distribute gdbus-generated sources

These are supposed to be regenerated at build time,
to avoid dependencies on the dist-systems glib version.

defaultvalue test: ignore show-desktop

Ignore the "show-desktop" property on GtkPlacesSidebar for the
defaultvalue test.

Currently, "make check" is passing because it runs the test under a xvfb
with no XSETTINGS provider, so we see the Gtk default value.  No matter
what we set the default value to in Gtk, however, there will be some
desktop environment in which someone running the installed test outside
of an xvfb will get the wrong result.  Best to ignore it.

https://bugzilla.gnome.org/show_bug.cgi?id=712302

Change the default for "show-desktop" back to TRUE

Change the GtkSettings default for "shell-shows-desktop" back to TRUE
and also change the default value of the "show-desktop" property on
GtkPlacesSidebar so that the defaultvalue test passes.

https://bugzilla.gnome.org/show_bug.cgi?id=712302

bindings: Fix stray semicolons in CSS example for bindings

Semicolon shouldn't be at the end of binding set

x11: Don't crash if a window is on an unmanaged screen

_gdk_x11_display_screen_for_xrootwin() can return NULL. If so, return
NULL from gdk_x11_window_foreign_new_for_display().

https://bugzilla.gnome.org/show_bug.cgi?id=709716

x11: keep track of the screen pixel size by calculating the bounding box of monitors

This is so we always have the latest information given by XRandR (or other), and not
rely on Core protocol information that might not have been updated yet.

This is specially visible when a monitor is connected (less frequent) or disconnected
(much more frequent), callbacks on GdkScreen::monitors-changed that call
gdk_screen_get_width/height() could get the screen size previous to the monitor
rearrangement.

So in order to fix this, keep track of the latest monitors information, and calculate
the bounding box in order to know the screen size.

https://bugzilla.gnome.org/show_bug.cgi?id=715029

printing: Fix memory leak when listing Avahi printers

Return values of g_variant_get_child_value() were not unreffed
correctly together with one value returned by g_variant_get().
Use g_variant_get_data() instead of copying each byte separately.

https://bugzilla.gnome.org/show_bug.cgi?id=712799

Broadway - Fix UNIX/Win32 detection

Finally fix this check so unnecessary headers aren't needed.

tests: Add max-width-chars to testwindowsize test

menu: Do a proper min height request

The minimum height is the size of one menu item plus the arrows, not the
minimum size of all menu items.

https://bugzilla.gnome.org/show_bug.cgi?id=691974
https://bugzilla.gnome.org/show_bug.cgi?id=694859

window: Don't assume 640x480 max default size

Instead, use the monitor's work area.

This might have unforseen side effects that warrant a later revert, such
as:
- Apparently some WMs assume maximizing when a window is maximum screen
  size.
- WMs might not shrink the window by the decorations' size when it tries
  to be fullscreen.
- Applications might have buggy size request code that causes weirdly
  sized windows.

stack: Fix crash on g_object_get("visible-child")

priv->visible_child is not a GtkWidget but a custom struct holding the
widget. So use the getter instead.

treemenu: Implement height-for-width properly

testsuite: Fix potential strcmp() against NULL

Use g_strcmp0() instead.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeviewaccessible: Fix a potential division by zero

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gdkwindow: Fix potential uses of uninitialised variables

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

pixbuf-engine: Clean up error conditions and destructors

Simplify the error checks and move all common behaviour into a utility
function.

https://bugzilla.gnome.org/show_bug.cgi?id=712536

pixbuf-engine: Improve ThemePixbuf clean up functions

Make theme_pixbuf_destroy() NULL-safe like g_free(), and add a clear
function in the spirit of the g_clear_* family of functions.

https://bugzilla.gnome.org/show_bug.cgi?id=712536

Themes with specified gap images but no actual file crash.

https://bugzilla.gnome.org/show_bug.cgi?id=712536

x11: Unset GdkX11DeviceXI2 scroll valuators on device change

Scroll valuators were being just appended again and again, leading
to 1) a growing memory issue anytime a device changed 2) the first
scroll valuators to stay permanent on the application lifetime, as
the first stored valuators would always match.

https://bugzilla.gnome.org/show_bug.cgi?id=705203

build: bump requirement on gobject-introspection to 1.39.0

https://bugzilla.gnome.org/show_bug.cgi?id=713044

wayland: Sync transient-for on xdg-surface show as well...

wayland: Allow set_title after initial showing

and fix the ordering of title / app_id

Update xdg-shell.xml

printing: Don't wake up Avahi printers unnecessarily

Use info available in Avahi TXT records for creation of gtk printer
and request details when needed (through gtk_printer_request_details()).
If there is a printer advertised on Avahi by a remote CUPS server
try to get its PPD file at first or get its capabilities through an IPP
request if it fails.

https://bugzilla.gnome.org/show_bug.cgi?id=712751

combobox: Set GDK_SCROLL_MASK on the eventbox in appears-as-list mode

This is so the scroll wheel works in both modes, and there's no behavioral
change depending on a style property.

https://bugzilla.gnome.org/show_bug.cgi?id=702663

Update xdg-shell.xml

wayland: Don't assert fail in DND

This needs completion, sure thing, but let's try to just not fall
flat on our face first.

wayland: Don't recreate the gtk_surface on every show

It's illegal.

Revert "stack: respect gtk-enable-animations setting"

It already respected enable-animations.

This reverts commit fb9ff442997f02ee2d6ad0eae0321a961e96f56f.

wayland: Set DBus properties after we've constructed the xdg_surface

wayland: Fix invalid cast in transient_for

treeview: Use CSS property "border-top-color" to draw grid lines

This allows themes to customize the grid line colors in treeviews.

https://bugzilla.gnome.org/show_bug.cgi?id=415415

stack: respect gtk-enable-animations setting

https://bugzilla.gnome.org/show_bug.cgi?id=712632

revealer: Respect gtk-enable-animations setting

https://bugzilla.gnome.org/show_bug.cgi?id=712632

Bump version

3.11.2

reftests: Fix rendering issues with border-image reftest

When downscaling images, Cairo apparently uses algorithms different
enough to make this test trip over. So add the downscaled image as the
reference instead of downscaling the previous reference image.

Fixes the border-image reftest. For real now.

reftests: Fix border-image-repeat test

The new downscaling code in Cairo doesn't allow this test, so we remove
the CSS that made the border downscaled.

So the test does test less now, but it still tests the repeat modes of
border images.

gdk: Ignore crossings generated by passive grabs when resetting scroll axes

Passive grabs may take pointer focus out of the application, even though
the pointer didn't leave the window, but those events still trigger resetting
of the scroll axes. This is most visible with compiz, and possibly other
reparenting WMs, where passive grabs happen on the WM-managed window that
is a parent of the application toplevel.

As it is not possible to have scrolling happening on the timespan a passive
grab takes action, it is entirely safe for GTK+ to assume none happened if
it gets a crossing event of that nature.

https://bugzilla.gnome.org/show_bug.cgi?id=699574#c33

wayland: Replace wl_shell_surface with xdg_shell

Fix missing return and closing brace.

https://bugzilla.gnome.org/show_bug.cgi?id=712514

GtkBuilder: Make IDs optional

One requirement of .ui files is that each object must have an ID,
even if it is never referred to or directly loaded from the code.
This makes editing .ui files much more onerous than it has to be,
due to the frequent need to invent new IDs, while avoiding
clashes.

This commit makes IDs optional in the XML. They only need to
be provided for objects which are referred to or explictly loaded
from the code. Since GtkBuilder needs IDs for its own internal
accounting, we create IDs of the form ___object_N___ if not
specified in the XML.

https://bugzilla.gnome.org/show_bug.cgi?id=712553

Document GTK_THEME

Fix doc build

gtkwindow: Fix cast warning

gtkwidget: Fix up some deprecation warnings

Updated POTFILES.skip

x11: Add extra debug spew on XI2 crossing and button events

This information will be useful in case someone stumbles on a situation
similar to https://bugzilla.gnome.org/show_bug.cgi?id=699574, so we can
figure out where do the crossing events come from or go to easily.

gdk: be more selective resetting scroll events

It's been reported in several applications that scrolling feels jerky
since commit cc7b3985b3e31.

Investigation reported that the combination of passive 4-7 button grabs
on the toplevel and the presence of native subwindows might trigger
too often crossing events from the child window to the toplevel and
back as scroll "buttons" trigger the passive grab. Those crossing events
would reset the scroll valuators rendering scrolling from jerky on
touchpads (where there's intermediate smooth events between the emulated
button ones) to ineffective on regular mouse wheels (where the crossing
event would reset the valuators right before the single smooth scroll
event we get is delivered)

So, only reset scroll valuators when the pointer enters the toplevel
(we only care about this when the pointer is on the window after it's
been possibly scrolling somewhere else), and it doesn't come from an
inferior.

The situations where this happened varied though, the native subwindow
could be one created explicitly by the application, or created indirectly
through gdk_window_ensure_native(). The latter was mainly the case for
evolution (through gtk_selection_set_owner()) and any GtkScrolledWindow
under the oxygen-gtk3 theme (through gdk_window_set_composited())

https://bugzilla.gnome.org/show_bug.cgi?id=699574

Use window icons for fallback

When the window has an icon, use it for the fallback menu
button.

Add the new exampleapp to the docs

Add another example application

This new step sets the headerbar as titlebar, and enables
fallback for the app menu.

Small doc improvements

Fix fallback menubar size allocation

When the menubar inserted by GtkApplicationWindow is the widest
widget in a csd window, its allocation gets cut short. Fix this
by taking the decoration size into account while calculating
the size request (it is implicitly taken into account in the
size allocation phase by _gtk_window_set_allocation).

GtkWindow: better app menu fallback for CSD

Do the menubutton for app menu fallback ourselves in GtkWindow
for the csd, non-custom titlebar case. This fits better with
the way we handle other title buttons. Themes have control
over the placement of this button by placing menu in the
decoration-button-layout style property.

Trivial cleanup

Trivial whitespace fix

Updated Polish translation

Updated Spanish translation

Updated Spanish translation

Test headerbar app menu fallback

Make the sunny example useful by giving it a header bar
with app menu fallback. To test this under gnome-shell,
set APP_MENU_FALLBACK=1 in the environment.

GtkApplicationWindow: Avoid double fallback for the app menu

If we have a header bar which has the app menu fallback
enabled, don't add the app menu to the fallback menubar.

GtkHeaderBar: Add a fallback app menu implementation

Allow showing the fallback app menu with a menu button
in the header bar. Applications have to explicitly enable
this by calling gtk_header_bar_set_show_fallback_app_menu.

GtkWindow: Make titlebar widget available privately

This will be used in GtkApplicationWindow in the
following commits.

GtkButton: don't propagate double-clicks

It is basically always unexpected to have a container handle
double-clicks on a widget that handles single-clicks itself.

Updated Greek translation

Updated Brazilian Portuguese translation

tests: Make testinput example not compress events

testgtk: Don't use GDK_POINTER_MOTION_HINT_MASK anymore

gdk: Document event compression better

Advise against using GDK_POINTER_MOTION_HINT_MASK.
Add pointers to gdk_window_set_event_compression() to the documentation.